Skip to content

Conversation

@pokornyd
Copy link
Member

@pokornyd pokornyd commented Apr 8, 2025

Motivation

Disables test parallelization because tests fail intermittently due to static nature of Console.Out, which sometimes gets polluted by strings from simultaneously ran tests, thus failing other tests.

XUnit doesn't seem to have a [TextFixtureSetup] equivalent, so disabling parallelization is a cost-effective, albeit hacky solution that doesn't involve rewriting the tests completely or making ArgHelpers class non-static.

Checklist

  • Code follows coding conventions held in this repo
  • Automated tests have been added
  • Tests are passing
  • Docs have been updated (if applicable)
  • Temporary settings (e.g. variables used during development and testing) have been reverted to defaults

How to test

If manual testing is required, what are the steps?

@pokornyd pokornyd requested a review from a team as a code owner April 8, 2025 15:49
@pokornyd
Copy link
Member Author

pokornyd commented Apr 8, 2025

shameless review bypass ✅

@pokornyd pokornyd merged commit c542180 into master Apr 8, 2025
1 check passed
@pokornyd pokornyd deleted the test-fix branch April 8, 2025 15:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants